Skip to content

[Backport 2.x] Adds saml auth header to differentiate saml requests and prevents auto login as anonymous user when basic authentication fails#4228

Merged
peternied merged 1 commit into2.xfrom
backport/backport-4152-to-2.x
Apr 9, 2024
Merged

[Backport 2.x] Adds saml auth header to differentiate saml requests and prevents auto login as anonymous user when basic authentication fails#4228
peternied merged 1 commit into2.xfrom
backport/backport-4152-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport d87ab3f from #4152.

…o login as anonymous user when basic authentication fails (#4152)

Signed-off-by: Darshit Chanpura <dchanp@amazon.com>
(cherry picked from commit d87ab3f)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 65.71%. Comparing base (4b6815f) to head (b9a0638).
Report is 1 commits behind head on 2.x.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              2.x    #4228      +/-   ##
==========================================
- Coverage   65.72%   65.71%   -0.02%     
==========================================
  Files         295      295              
  Lines       21085    21089       +4     
  Branches     3460     3461       +1     
==========================================
  Hits        13859    13859              
- Misses       5490     5491       +1     
- Partials     1736     1739       +3     
Files Coverage Δ
...g/opensearch/security/rest/SecurityInfoAction.java 52.72% <100.00%> (+0.87%) ⬆️
.../org/opensearch/security/auth/BackendRegistry.java 61.25% <46.15%> (-1.29%) ⬇️

... and 1 file with indirect coverage changes

@peternied peternied merged commit 7b05568 into 2.x Apr 9, 2024
@peternied peternied deleted the backport/backport-4152-to-2.x branch April 9, 2024 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants